feat(RELEASE-1147): update repository to registry.redhat.io in rpa #1401
Conversation
This commit change the repository value from quay* to registry* in RPA mapping. This was implemented in release-238. Signed-off-by: Happy Bhati <[email protected]>
| releasecommon "github.com/konflux-ci/e2e-tests/tests/release" | ||
| releaseapi "github.com/konflux-ci/release-service/api/v1alpha1" | ||
| tektonutils "github.com/konflux-ci/release-service/tekton/utils" | ||
| tektonv1 "github.com/tektoncd/pipeline/pkg/apis/pipeline/v1" | ||
| corev1 "k8s.io/api/core/v1" | ||
| metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" |
There was a problem hiding this comment.
It seems to me that these imports were alphabetically ordered, but now they're not. Why this change?
There was a problem hiding this comment.
Format on save on the VS-Code did the auto-formatting when I saved the file. In all the files I need to disable this sometimes it formats unwanted lines.
There was a problem hiding this comment.
ok, it's fine with me. but I'd like @jinqi7 's input on this, since she's the original author of the test.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: johnbieren The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This commit change the repository value from quay* to registry*
in RPA mapping. This was implemented in release-238.
Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Issue ticket number and link
Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: